Skip to content
This repository has been archived by the owner on Oct 16, 2023. It is now read-only.

Fix a bug in BatchNorm #130

Merged
merged 1 commit into from
Sep 15, 2023
Merged

Fix a bug in BatchNorm #130

merged 1 commit into from
Sep 15, 2023

Conversation

kakao-steve-ai
Copy link
Contributor

@kakao-steve-ai kakao-steve-ai commented Sep 15, 2023 •
edited
Loading

πŸ™ Describe the pull request

FP16 νƒ€μž…μ— λŒ€ν•œ Backward 였λ₯˜ μˆ˜μ •ν–ˆμŠ΅λ‹ˆλ‹€.

βœ… Checklist

  • Code follows the project's coding conventions and style.
  • Tests have been added or updated to cover the changes.
  • Documentation has been updated, if necessary.

mejai1206
mejai1206 approved these changes Sep 15, 2023
View reviewed changes
Copy link
Contributor

@mejai1206 mejai1206 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

daemyung
daemyung approved these changes Sep 15, 2023
View reviewed changes
Copy link
Contributor

@daemyung daemyung left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kakao-steve-ai kakao-steve-ai merged commit 7287187 into main Sep 15, 2023
1 check passed
@kakao-steve-ai kakao-steve-ai deleted the batch_norm branch September 15, 2023 06:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@daemyung daemyung daemyung approved these changes

@mejai1206 mejai1206 mejai1206 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@kakao-steve-ai @daemyung @mejai1206